Skip to content

remove that tailwind import#1780

Merged
carlosabadia merged 1 commit intomainfrom
remove-that-tailwind-import
Mar 17, 2026
Merged

remove that tailwind import#1780
carlosabadia merged 1 commit intomainfrom
remove-that-tailwind-import

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 17, 2026

Greptile Summary

This PR removes the redundant @import "tailwindcss"; line from assets/tailwind-theme.css. Since rxconfig.py already registers rx.plugins.TailwindV4Plugin(), Tailwind's core styles are automatically injected by the plugin — making the manual CSS import unnecessary and potentially causing duplicate style injection.

  • Removes @import "tailwindcss"; from assets/tailwind-theme.css
  • The TailwindV4Plugin in rxconfig.py handles Tailwind CSS injection, so this import was redundant
  • @import "tailwindcss-animated" is retained since that plugin is not managed by TailwindV4Plugin directly
  • No functional behavior is changed; this is a cleanup to avoid potential duplicate/conflicting CSS imports

Confidence Score: 5/5

  • This PR is safe to merge — it removes a single redundant CSS import with no risk of breaking functionality.
  • The change is a one-line deletion of a CSS import that is already handled by the registered TailwindV4Plugin. No logic, state, or component behavior is affected. The remaining imports and custom variables in the file are untouched.
  • No files require special attention.

Important Files Changed

Filename Overview
assets/tailwind-theme.css Removes the redundant @import "tailwindcss" line, which is already handled by the TailwindV4Plugin registered in rxconfig.py. Clean and safe removal.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["rxconfig.py\nrx.plugins.TailwindV4Plugin()"] -->|"automatically injects"| B["Tailwind CSS core styles"]
    C["assets/tailwind-theme.css"] -->|"imports"| D["tailwindcss-animated"]
    C -->|"defines"| E["Custom CSS variables & dark variant"]
    B --> F["Final CSS bundle"]
    D --> F
    E --> F
    style A fill:#6d28d9,color:#fff
    style C fill:#6d28d9,color:#fff
Loading

Last reviewed commit: 5214453

@carlosabadia carlosabadia merged commit 0aa2fd1 into main Mar 17, 2026
10 checks passed
@carlosabadia carlosabadia deleted the remove-that-tailwind-import branch March 17, 2026 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants